Fix fullscreen-options wpt test The test failed because a promise was uncaught. Ensure that either a rejected or allowed promise will allow the test to pass provided Change-Id: I604ab23fe32a1dee1ed60182dd25e4ffbd60a688 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548032 Reviewed-by: Robert Ma <robertma@chromium.org> Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#646438} diff --git a/fullscreen/api/element-request-fullscreen-options.html b/fullscreen/api/element-request-fullscreen-options.html index c6ce1fd..6a0bfa1 100644 --- a/fullscreen/api/element-request-fullscreen-options.html +++ b/fullscreen/api/element-request-fullscreen-options.html
@@ -8,11 +8,14 @@ // no normative requirements on what navigationUI should do, just test for // basic support. (One could also check that the three allowed enum valid are // supported and no others, but that would overlap with UA-specific tests.) -test(() => { +promise_test(() => { let invoked = false; - document.body.requestFullscreen({ + return document.body.requestFullscreen({ get navigationUI() { invoked = true; return "irrelevant-value"; } + }).then(() => { + assert_unreached("promise should be rejected due to invalid navigationUI value"); + }, () => { + assert_true(invoked, "navigationUI getter invoked"); }); - assert_true(invoked, "navigationUI getter invoked"); }); </script> 